Skip to content

Conversation

@astroshim
Copy link
Contributor

What is this PR for?

If you run the following code, then streaming output doesn't work properly from the second run.

%spark.pyspark
import time
print("1")
time.sleep(2)
print("2")
time.sleep(2)
print("3")
time.sleep(2)
print("4")

This problem comes from the order of paragraph update event timing and paragraph update-append event timing is incorrect.
and This PR will fix also #1833 too.

What type of PR is it?

Bug Fix

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1994

How should this be tested?

  • run several times pyspark interpreter with above code.

Screenshots (if appropriate)

  • before
    2017-01-21 00_55_25

  • after
    2017-01-21 00_59_12

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions?no
  • Does this needs documentation?no

@AhyoungRyu
Copy link
Contributor

AhyoungRyu commented Jan 21, 2017

I tested #1833 upon this patch and #1922, it's perfectly working now. Appreciate for fixing this!
👍 x 100

@Leemoonsoo
Copy link
Member

Leemoonsoo commented Jan 21, 2017

Great fix. LGTM.
Merge to master and branch-0.7 when no more comments.

@asfgit asfgit closed this in 1a1fbc4 Jan 23, 2017
asfgit pushed a commit that referenced this pull request Jan 23, 2017
### What is this PR for?
If you run the following code, then streaming output doesn't work properly from the second run.
```
%spark.pyspark
import time
print("1")
time.sleep(2)
print("2")
time.sleep(2)
print("3")
time.sleep(2)
print("4")
```
This problem comes from the order of `paragraph update` event timing and `paragraph update-append` event timing is incorrect.
and This PR will fix also #1833 too.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1994

### How should this be tested?
- run several times pyspark interpreter with above code.

### Screenshots (if appropriate)
- before
![2017-01-21 00_55_25](https://cloud.githubusercontent.com/assets/3348133/22173437/bfa48e64-df77-11e6-9625-ab44dedee395.gif)

- after
![2017-01-21 00_59_12](https://cloud.githubusercontent.com/assets/3348133/22173438/c21820ac-df77-11e6-87dc-07970fca13ca.gif)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions?no
* Does this needs documentation?no

Author: astroshim <[email protected]>

Closes #1927 from astroshim/ZEPPELIN-1994 and squashes the following commits:

c7baa59 [astroshim] fix streaming output problem

(cherry picked from commit 1a1fbc4)
Signed-off-by: ahyoungryu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants